Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor view #14

Draft
wants to merge 23 commits into
base: master
Choose a base branch
from
Draft

Refactor view #14

wants to merge 23 commits into from

Conversation

caleb15
Copy link
Contributor

@caleb15 caleb15 commented Aug 11, 2021

see https://github.com/15five/dev-discussions/issues/117

RELEASES
Closes DEV-1432 Change presentation of release notes

caleb15 added 16 commits July 29, 2021 17:23
darn ImportError...
okay actually 97% but close enough
Seems useful to list env to distuingish release from other releases. Also we don't need a release number at the moment. It would take extra work to do note_format so not doing at the moment. Renaming JIRA_TICKETS_NUM to NUM_TICKETS so we it applies even if we switch away from jira.
no longer used
@caleb15 caleb15 marked this pull request as draft August 11, 2021 23:22
[
"github token",
"0782415",
"8038fc3",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These shas were from https://github.com/15five/fifteen5-deploy, I updated them to shas from this repo

also do raise for status so future errors dont happen silently
the header is plaintext so I shouldn't do markdown
for label in labels:
if label.startswith(category_indicator):
return label.replace(category_indicator, "")
return "Uncategorized"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Totally forgot about https://github.com/15five/dev-discussions/issues/117#issuecomment-839225813 -_-

I'll need to add a new thing to extra category from the Jira ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant